x86 xsave: Adding back CPUID support for Xsave (version 2)

XSave support via CPUID virtualization for both PV and HVM guests.

Signed-off-by: Shan Haitao <haitao.shan@intel.com>

diff --git a/tools/libxc/xc_cpuid_x86.c b/tools/libxc/xc_cpuid_x86.c
index e517eb7733112227abd2b845f101307dfe613f7d..7d963a903e515e9beda71c7dd6fc63b81a0dc5a3 100644 (file)
--- a/tools/libxc/xc_cpuid_x86.c
+++ b/tools/libxc/xc_cpuid_x86.c
@@ -168,13 +168,21 @@ static void xc_cpuid_hvm_policy(
     xc_interface *xch, domid_t domid,
     const unsigned int *input, unsigned int *regs)
 {
+    DECLARE_DOMCTL;
     char brand[13];
     unsigned long pae;
-    int is_pae;
+    int is_pae, xsave_supported;
 
     xc_get_hvm_param(xch, domid, HVM_PARAM_PAE_ENABLED, &pae);
     is_pae = !!pae;
 
+    /* Detecting Xen's atitude towards XSAVE */
+    memset(&domctl, 0, sizeof(domctl));
+    domctl.cmd = XEN_DOMCTL_getvcpuextstate;
+    domctl.domain = domid;
+    do_domctl(xch, &domctl);
+    xsave_supported = (domctl.u.vcpuextstate.xfeature_mask != 0);
+
     switch ( input[0] )
     {
     case 0x00000000:
@@ -195,7 +203,10 @@ static void xc_cpuid_hvm_policy(
                     bitmaskof(X86_FEATURE_SSE4_1) |
                     bitmaskof(X86_FEATURE_SSE4_2) |
                     bitmaskof(X86_FEATURE_POPCNT) |
-                    bitmaskof(X86_FEATURE_AES));
+                    bitmaskof(X86_FEATURE_AES) |
+                    (xsave_supported)?
+                     (bitmaskof(X86_FEATURE_AVX) |
+                      bitmaskof(X86_FEATURE_XSAVE)) : 0);
 
         regs[2] |= bitmaskof(X86_FEATURE_HYPERVISOR);
 
@@ -280,7 +291,7 @@ static void xc_cpuid_pv_policy(
     const unsigned int *input, unsigned int *regs)
 {
     DECLARE_DOMCTL;
-    int guest_64bit, xen_64bit = hypervisor_is_64bit(xch);
+    int guest_64bit, xsave_supported, xen_64bit = hypervisor_is_64bit(xch);
     char brand[13];
 
     xc_cpuid_brand_get(brand);
@@ -291,6 +302,13 @@ static void xc_cpuid_pv_policy(
     do_domctl(xch, &domctl);
     guest_64bit = (domctl.u.address_size.size == 64);
 
+    /* Detecting Xen's atitude towards XSAVE */
+    memset(&domctl, 0, sizeof(domctl));
+    domctl.cmd = XEN_DOMCTL_getvcpuextstate;
+    domctl.domain = domid;
+    do_domctl(xch, &domctl);
+    xsave_supported = (domctl.u.vcpuextstate.xfeature_mask != 0);
+
     if ( (input[0] & 0x7fffffff) == 1 )
     {
         clear_bit(X86_FEATURE_VME, regs[3]);
@@ -320,10 +338,14 @@ static void xc_cpuid_pv_policy(
         clear_bit(X86_FEATURE_TM2, regs[2]);
         if ( !guest_64bit )
             clear_bit(X86_FEATURE_CX16, regs[2]);
+        if ( !xsave_supported )
+        {
+            clear_bit(X86_FEATURE_XSAVE, regs[2]);
+            clear_bit(X86_FEATURE_AVX, regs[2]);
+        }
         clear_bit(X86_FEATURE_XTPR, regs[2]);
         clear_bit(X86_FEATURE_PDCM, regs[2]);
         clear_bit(X86_FEATURE_DCA, regs[2]);
-        clear_bit(X86_FEATURE_XSAVE, regs[2]);
         set_bit(X86_FEATURE_HYPERVISOR, regs[2]);
         break;
     case 0x80000001:

diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c
index 3924478eddbd2fabc5f0f007b29c95b1f190f978..966ec831d2424b2fc5ca4f63f7d991a7817f99c4 100644 (file)
--- a/xen/arch/x86/hvm/hvm.c
+++ b/xen/arch/x86/hvm/hvm.c
@@ -2089,15 +2089,10 @@ void hvm_cpuid(unsigned int input, unsigned int *eax, unsigned int *ebx,
         if ( vlapic_hw_disabled(vcpu_vlapic(v)) )
             __clear_bit(X86_FEATURE_APIC & 31, edx);
 
-        /* Fix up XSAVE and OSXSAVE. */
-        *ecx &= ~(bitmaskof(X86_FEATURE_XSAVE) |
-                  bitmaskof(X86_FEATURE_OSXSAVE));
+        /* Fix up OSXSAVE. */
         if ( cpu_has_xsave )
-        {
-            *ecx |= bitmaskof(X86_FEATURE_XSAVE);
             *ecx |= (v->arch.hvm_vcpu.guest_cr[4] & X86_CR4_OSXSAVE) ?
                      bitmaskof(X86_FEATURE_OSXSAVE) : 0;
-        }
         break;
     case 0xb:
         /* Fix the x2APIC identifier. */

diff --git a/xen/arch/x86/traps.c b/xen/arch/x86/traps.c
index 69bbe43cbb2971b31ef955d61fdee543831191aa..77af294f089e2981a77432bc1072ff3ae53022bb 100644 (file)
--- a/xen/arch/x86/traps.c
+++ b/xen/arch/x86/traps.c
@@ -795,7 +795,11 @@ static void pv_cpuid(struct cpu_user_regs *regs)
         __clear_bit(X86_FEATURE_XTPR % 32, &c);
         __clear_bit(X86_FEATURE_PDCM % 32, &c);
         __clear_bit(X86_FEATURE_DCA % 32, &c);
-        __clear_bit(X86_FEATURE_XSAVE % 32, &c);
+        if ( !cpu_has_xsave )
+        {
+            __clear_bit(X86_FEATURE_XSAVE % 32, &c);
+            __clear_bit(X86_FEATURE_AVX % 32, &c);
+        }
         if ( !cpu_has_apic )
            __clear_bit(X86_FEATURE_X2APIC % 32, &c);
         __set_bit(X86_FEATURE_HYPERVISOR % 32, &c);

diff --git a/xen/include/asm-x86/cpufeature.h b/xen/include/asm-x86/cpufeature.h
index 3ae6c786b7f06c7fb66a265ce2d039dc2670c1e6..e12785e891f6c013685121eeebee6a83ec3e2cc3 100644 (file)
--- a/xen/include/asm-x86/cpufeature.h
+++ b/xen/include/asm-x86/cpufeature.h
@@ -102,6 +102,7 @@
 #define X86_FEATURE_TSC_DEADLINE (4*32+24) /* "tdt" TSC Deadline Timer */
 #define X86_FEATURE_XSAVE      (4*32+26) /* XSAVE/XRSTOR/XSETBV/XGETBV */
 #define X86_FEATURE_OSXSAVE    (4*32+27) /* OSXSAVE */
+#define X86_FEATURE_AVX        (4*32+28) /* Advanced Vector Extensions */
 #define X86_FEATURE_HYPERVISOR (4*32+31) /* Running under some hypervisor */
 
 /* VIA/Cyrix/Centaur-defined CPU features, CPUID level 0xC0000001, word 5 */